Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

then-able dispatches to AliasedActions #309

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

PaulSzymanski
Copy link

Hey folks, I want to await dispatches in the renderer as I am able to in the main process.
I replaced the IPC send with an invoke-able IPC.
This works like a charm for me :)

I would love to hear your thoughts on this!

I did not adjust the type definitions yet.

@CLAassistant
Copy link

CLAassistant commented May 28, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@matmalkowski
Copy link
Collaborator

Hi @PaulSzymanski
await-able dispatches are not intended part of this library, and we don't plan to support them in v2 version

In the main process dispatches to thunk actions could be awaited.
However this was not possible for aliased thunk actions.
With this commit we can await dispatches to aliased thunk actions in the
renderer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants